Skip to content

Feat: support new Runtime Fuse CleanPolicy - OnFuseChanged#4800

Merged
fluid-e2e-bot[bot] merged 1 commit intofluid-cloudnative:masterfrom
Syspretor:feature/support-fuse-updateStategy-onIdle
Apr 16, 2025
Merged

Feat: support new Runtime Fuse CleanPolicy - OnFuseChanged#4800
fluid-e2e-bot[bot] merged 1 commit intofluid-cloudnative:masterfrom
Syspretor:feature/support-fuse-updateStategy-onIdle

Conversation

@Syspretor
Copy link
Copy Markdown
Collaborator

WIP: A proposal will be related later

@fluid-e2e-bot
Copy link
Copy Markdown

fluid-e2e-bot Bot commented Mar 21, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 9 times, most recently from 2e7c3ad to 1f4fb3a Compare March 27, 2025 03:00
@Syspretor Syspretor marked this pull request as ready for review March 27, 2025 03:20
Comment thread pkg/ddc/juicefs/sync_runtime.go
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch from 1f4fb3a to 3f620e7 Compare March 27, 2025 05:54
@cheyang
Copy link
Copy Markdown
Collaborator

cheyang commented Mar 27, 2025

/test-fluid-e2e

@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch from 3f620e7 to 42da4e9 Compare March 28, 2025 02:09
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 10 times, most recently from 8b1219d to 12f77b1 Compare April 2, 2025 02:13
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 3 times, most recently from 27172af to 39abd44 Compare April 9, 2025 11:58
Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SonarCloud found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch from 39abd44 to 0009daa Compare April 9, 2025 12:02
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 2 times, most recently from f10d123 to 92b7d26 Compare April 10, 2025 01:53
Comment thread api/v1alpha1/constant.go Outdated
Comment thread api/v1alpha1/constant.go Outdated
Comment thread api/v1alpha1/juicefsruntime_types.go Outdated
Comment thread pkg/csi/plugins/nodeserver.go Outdated
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 3 times, most recently from 26633ec to b08dbed Compare April 10, 2025 03:39
@Syspretor Syspretor requested review from cheyang and zwwhdls April 10, 2025 03:44
Comment thread api/v1alpha1/juicefsruntime_types.go Outdated
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch 2 times, most recently from 55984c1 to e36f287 Compare April 14, 2025 08:33
@Syspretor Syspretor changed the title Feat: support Runtime Fuse UpdateStrategy - OnIdle Feat: support new Runtime Fuse CleanPolicy - OnFuseUpdate Apr 15, 2025
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch from e36f287 to 4df051f Compare April 15, 2025 06:27
Signed-off-by: 玖宇 <guotongyu.gty@alibaba-inc.com>
@Syspretor Syspretor force-pushed the feature/support-fuse-updateStategy-onIdle branch from 4df051f to d91139c Compare April 15, 2025 09:16
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@cheyang cheyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

Copy link
Copy Markdown
Collaborator

@cheyang cheyang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@fluid-e2e-bot
Copy link
Copy Markdown

fluid-e2e-bot Bot commented Apr 16, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cheyang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants